Skip to content

[feat] added next.js documentation -sample guide based on the next.js documentation#24402

Merged
dvdksn merged 5 commits intodocker:mainfrom
kristiyan-velkov:feat/next-js-guide
Mar 31, 2026
Merged

[feat] added next.js documentation -sample guide based on the next.js documentation#24402
dvdksn merged 5 commits intodocker:mainfrom
kristiyan-velkov:feat/next-js-guide

Conversation

@kristiyan-velkov
Copy link
Copy Markdown
Contributor

Description

Adds a Next.js language-specific guide under content/guides/nextjs/:

  • _index.md — Overview, prerequisites, what you’ll learn
  • containerize.mddocker init, multi-stage Dockerfile, standalone/export, production image
  • develop.md — Compose dev/prod setup, Compose Watch, hot reload
  • run-tests.md — Vitest/Jest and ESLint in containers
  • configure-github-actions.md — CI/CD with GitHub Actions → Docker Hub
  • deploy.md — Deploy to local Kubernetes (Docker Desktop)

Uses sample app: kristiyan-velkov/docker-nextjs-sample.

Related issues or tickets

None

Reviews

  • [x ] Technical review
  • Editorial review
  • Product review

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 18, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit ff3a1b2
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/69cb6d34ff00db000801a90f
😎 Deploy Preview https://deploy-preview-24402--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@craig-osterhout craig-osterhout added the status/review Pull requests that are ready for review label Mar 18, 2026
@craig-osterhout craig-osterhout requested a review from a team March 18, 2026 14:57
@dvdksn
Copy link
Copy Markdown
Contributor

dvdksn commented Mar 18, 2026

/review

Copy link
Copy Markdown
Contributor

@docker-agent docker-agent bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assessment: 🟡 NEEDS ATTENTION

This PR adds a comprehensive Next.js language-specific guide to Docker documentation. The content is technically accurate and well-structured, but contains several style guide violations that should be addressed before merging.

Summary:

  • ✅ All front matter is complete (title, description, keywords)
  • ✅ Code examples appear correct
  • ✅ No security issues detected
  • ⚠️ 4 style guide violations (hedge words and line length)

Issues found:

  • 3 instances of prohibited hedge words ("simplifies", "seamlessly", "efficiently")
  • 2 instances of prose lines exceeding 80 characters

These are documentation quality issues that affect readability and consistency with Docker's style guide.

Copy link
Copy Markdown
Contributor

@brandonh6k brandonh6k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, looks good. :shipit:

And I understand on the pinning - I can see both sides of the argument. Honestly, I don't know that there's a "right" answer here - it all comes down to a matter of taste and exposure, so either way is correct, depending on your risk tolerance and use case.

kristiyan-velkov and others added 2 commits March 31, 2026 09:43
Co-authored-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
Co-authored-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
@dvdksn dvdksn merged commit ffcbd75 into docker:main Mar 31, 2026
29 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/guides status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants